Skip to content

chore(dynamic-sampling): cleanup transaction based health check rule#115471

Merged
shellmayr merged 3 commits into
masterfrom
shelmayr/chore/remove-transaction-based-health-check-bias
May 18, 2026
Merged

chore(dynamic-sampling): cleanup transaction based health check rule#115471
shellmayr merged 3 commits into
masterfrom
shelmayr/chore/remove-transaction-based-health-check-bias

Conversation

@shellmayr
Copy link
Copy Markdown
Member

  • We've dialed in the new trace based health check bias to end up roughly with the same volume as the previous transaction based ruleset
  • Clean up the rule & all associated artifacts

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 13, 2026
@shellmayr shellmayr marked this pull request as ready for review May 13, 2026 09:41
@shellmayr shellmayr requested review from a team as code owners May 13, 2026 09:41
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9094659. Configure here.

Comment thread tests/sentry/dynamic_sampling/test_generate_rules.py
Copy link
Copy Markdown
Member

@Dav1dde Dav1dde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice

@shellmayr shellmayr merged commit caaa13d into master May 18, 2026
83 checks passed
@shellmayr shellmayr deleted the shelmayr/chore/remove-transaction-based-health-check-bias branch May 18, 2026 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants